Skip to content

feat: handle LinkedEvents API timeouts for popular keywords#454

Merged
mhieta merged 1 commit into
mainfrom
seer/feat/handle-keyword-api-timeouts-SWDmNi
Jan 22, 2026
Merged

feat: handle LinkedEvents API timeouts for popular keywords#454
mhieta merged 1 commit into
mainfrom
seer/feat/handle-keyword-api-timeouts-SWDmNi

Conversation

@seer-by-sentry

Copy link
Copy Markdown
Contributor

Fixes KULTUS-API-M. The issue was that: GraphQL resolver resolve_popular_kultus_keywords lacks error handling for ConnectTimeoutError during sequential external API calls, causing query failure.

  • Added error handling for requests.exceptions.ConnectTimeout, requests.exceptions.Timeout, and requests.exceptions.RequestException when fetching keyword sets from the LinkedEvents API.
  • Ensures that the popularKultusKeywords query continues to process other keyword sets and returns partial results even if one API request fails.
  • Introduced new tests to verify graceful degradation and partial result handling in case of API timeouts or other request exceptions.

This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 572273

Not quite right? Click here to continue debugging with Seer.

@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr454.api.dev.hel.ninja 🚀🚀🚀

Add graceful degradation for LinkedEvents API timeout
handling in popularKultusKeywords resolver. The resolver now
continues processing remaining keyword sets even if one
request fails due to timeout or connection errors.

Refs: PT-1990
@mhieta mhieta force-pushed the seer/feat/handle-keyword-api-timeouts-SWDmNi branch from 1476529 to 72f74df Compare January 20, 2026 19:52
@mhieta mhieta changed the title feat: Handle LinkedEvents API timeouts for popular keywords feat: handle LinkedEvents API timeouts for popular keywords Jan 20, 2026
@sonarqubecloud

Copy link
Copy Markdown

@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr454.api.dev.hel.ninja 🚀🚀🚀

@mhieta mhieta marked this pull request as ready for review January 20, 2026 20:24
@mhieta mhieta requested a review from a team as a code owner January 20, 2026 20:24
@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr454.api.dev.hel.ninja 🚀🚀🚀

@mhieta mhieta merged commit 54dcdf7 into main Jan 22, 2026
26 checks passed
@mhieta mhieta deleted the seer/feat/handle-keyword-api-timeouts-SWDmNi branch January 22, 2026 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants